CW 882: Integrate Seed Verification Flow To Integration Tests #1966
Conversation
…Integrate-Seed-Verification-Flow-To-Integration-Tests
… test script to add a watch dog to prevent infinite running on CI
https://github.com/cake-tech/cake_wallet into Integrate-Seed-Verification-Flow-To-Integration-Tests
| static const int verificationWordsCount = 2; |
There was a problem hiding this comment.
why? we want this to skip verification in debug
let's find a way to make the tests run in release mode, if not, then we can find a different approach, maybe set it in an environment variable and use that or if you have something else in mind
or use the TestAssetBundle that you made
… and resulting errors
| //* ======Confirm it displays the properly formatted title=========== | ||
| final transactionType = dashboardViewModel.getTransactionType(item.transaction); | ||
|
|
||
| final title = item.formattedTitle + item.formattedStatus + transactionType; | ||
|
|
||
| commonTestCases.findWidgetViaDescendant( | ||
| of: find.byKey(ValueKey(keyId)), | ||
| matching: find.text(title), | ||
| ); |
There was a problem hiding this comment.
forgot to add a TODO there, there's been some inconsistencies with the wordings for the title, something from the flutter end as it doesn't constantly happen. Happens once, and when you rerun, it doesn't happen again. I think I mentioned earlier on one of the calls.
| final onAuthPage = authPageRobot.onAuthPage(); | ||
| if (onAuthPage) { | ||
| await authPageRobot.enterPinCode(CommonTestConstants.pin); | ||
| } | ||
|
|
||
| final onAuthPageDesktop = authPageRobot.onAuthPageDesktop(); | ||
| if (onAuthPageDesktop) { | ||
| await authPageRobot.enterPassword(CommonTestConstants.pin.join("")); | ||
| } |
There was a problem hiding this comment.
doesn't the exchange flow have the potential to show the auth page? why remove it then?
There was a problem hiding this comment.
doesn't the exchange flow have the potential to show the auth page? why remove it then?
It has already been handled in the Exchange flow robot, in the flow that handles all events post exchange.
I didn't add it in earlier, I think it was added with the desktop flow setup done earlier
Issue Number (if Applicable): Fixes #
Description
Please include a summary of the changes and which issue is fixed / feature is added.
Pull Request - Checklist